Skip to content

feat(Worklets): always compile bundle mode symbols#9064

Merged
tjzel merged 9 commits into
mainfrom
@tjzel/worklets/bundle-mode-symbols
Mar 16, 2026
Merged

feat(Worklets): always compile bundle mode symbols#9064
tjzel merged 9 commits into
mainfrom
@tjzel/worklets/bundle-mode-symbols

Conversation

@tjzel

@tjzel tjzel commented Mar 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR is a step that makes experimental Bundle Mode less experimental.

Having the use of Bundle Mode gated behind #ifdef macros is burdensome for prebuild environments or large scale apps.

This PR makes it so the native side receives the information whether to use the Bundle Mode from JavaScript, instead from static feature flags. All Bundle Mode symbols are available without re-compilation, resulting in much faster toggling of the feature.

The source of truth for whether the Bundle Mode is enabled is the babel.config.js - passing bundleMode: true in the config enables the bundle mode.

Test plan

Compiled and ran the example app both with and without the Bundle Mode.

@tjzel tjzel requested a review from Copilot March 13, 2026 18:27
@tjzel tjzel marked this pull request as ready for review March 13, 2026 18:31

This comment was marked as resolved.

Comment thread scripts/patches/bundle-mode.patch Outdated
@tjzel tjzel added this pull request to the merge queue Mar 16, 2026
Merged via the queue into main with commit adbbd15 Mar 16, 2026
26 checks passed
@tjzel tjzel deleted the @tjzel/worklets/bundle-mode-symbols branch March 16, 2026 13:49
github-merge-queue Bot pushed a commit that referenced this pull request Mar 20, 2026
## Summary

Updating the docs since `BUNDLE_MODE_ENABLED` is no longer a static
feature flag with #9064, also cleaning up a bit in the repo.

## Test plan

CI
tshmieldev pushed a commit that referenced this pull request Mar 24, 2026
## Summary

Updating the docs since `BUNDLE_MODE_ENABLED` is no longer a static
feature flag with #9064, also cleaning up a bit in the repo.

## Test plan

CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants